Skip to content

docs(tune): fix import count in CIFAR PyTorch tutorial#62756

Merged
justinvyu merged 2 commits into
ray-project:masterfrom
RudrenduPaul:docs/fix-cifar-imports-count
May 22, 2026
Merged

docs(tune): fix import count in CIFAR PyTorch tutorial#62756
justinvyu merged 2 commits into
ray-project:masterfrom
RudrenduPaul:docs/fix-cifar-imports-count

Conversation

@RudrenduPaul
Copy link
Copy Markdown
Contributor

@RudrenduPaul RudrenduPaul commented Apr 19, 2026

Why are these changes needed?

The CIFAR PyTorch Tune tutorial says:

Only the last three imports are for Ray Tune.

But the actual import cell only has two Ray Tune imports:

from ray import tune
from ray.tune.schedulers import ASHAScheduler

This updates the narrative text to match the actual code.

Related issue

Fixes #61250

Checks

  • I've signed off all commits with git commit -s (N/A — trivial text correction)
  • I've run the relevant tests for my changes
  • I've made sure the tests are passing

@RudrenduPaul
docs(tune): fix import count in CIFAR PyTorch example
6b84a46 
The CIFAR PyTorch tutorial imports only 2 Ray Tune items (`from ray
import tune` and `from ray.tune.schedulers import ASHAScheduler`),
not 3 as stated in the narrative text. Corrects the count to match
the actual code.

Fixes ray-project#61250
@RudrenduPaul RudrenduPaul requested a review from a team as a code owner April 19, 2026 05:25
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Apr 19, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the documentation in the PyTorch CIFAR example notebook to correctly reflect the number of Ray Tune imports. I have no feedback to provide.

@ray-gardener ray-gardener Bot added tune Tune-related issues docs An issue or change related to documentation community-contribution Contributed by the community labels Apr 19, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions Bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label May 3, 2026
pseudo-rnd-thoughts
pseudo-rnd-thoughts approved these changes May 6, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@pseudo-rnd-thoughts pseudo-rnd-thoughts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, LGTM

@pseudo-rnd-thoughts pseudo-rnd-thoughts added go add ONLY when ready to merge, run all tests and removed stale The issue is stale. It will be closed within 7 days unless there are further conversation labels May 6, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions Bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label May 21, 2026
@justinvyu justinvyu merged commit 28ec495 into ray-project:master May 22, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pseudo-rnd-thoughts pseudo-rnd-thoughts pseudo-rnd-thoughts approved these changes

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

community-contribution Contributed by the community docs An issue or change related to documentation go add ONLY when ready to merge, run all tests stale The issue is stale. It will be closed within 7 days unless there are further conversation tune Tune-related issues

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Docs: Incorrect count of Ray Tune imports in CIFAR PyTorch example

4 participants

@RudrenduPaul @pseudo-rnd-thoughts @justinvyu @matthewdeng